-
Notifications
You must be signed in to change notification settings - Fork 291
feat: refresh MessageComposer design #2919
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
| export const IconPause = ({ className, ...props }: ComponentProps<'svg'>) => ( | ||
| <svg | ||
| {...props} | ||
| className={clsx('str-chat__icon--pause', className)} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it would be good if all icons have a shared selector str-chat__icon so one can target all icons at once, instead of writing hundreds of individual class names.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will add a BaseIcon wrapper that will inject that class.
|
Size Change: +12.6 kB (+4.35%) Total Size: 303 kB
ℹ️ View Unchanged
|
87f4b53 to
a4ae4bc
Compare
🎯 Goal
Add new styles:
Add new functionalities coming with design refresh:
###? Uknown?
Out of scope
🛠 Implementation details
Provide a description of the implementation
🎨 UI Changes
Add relevant screenshots